I think this is ready for prime time. Allen has put this through the hoops here at threadsdev and after some heavy stomping I think we got the bugs out. <img src="/threads/php/images/icons/wink.gif" alt="" /><br /><br />This modification will display who made the last post in a thread that links to the related last post (simular to a .classic modification). This information will be displayed in the "last post" column of the postlist.php script. Also note that this was made to work with MySQL database only.<br /><br /><br />The following files will need to be modified:<br />__ postlist.php __________ (threads directory)<br />__ postlist.tmpl _________ (threads directory)<br />__ addpost.php ___________ (threads directory)<br />__ doapproveposts.php ____ (admin directory)<br />__ viewunapproved.php ____ (admin directory)<br />__ deletepost.php ________ (threads directory)<br /><br />This modification will require 3 additional fields to be added to your w3t_Posts table in your database. An "alter table" script is included to make these additions for you.<br /><br />The file is attached. Enjoy! <img src="/threads/php/images/icons/smile.gif" alt="" />

Thanks Dave! <img src="/forum/images/icons/smile.gif" alt="" /><br /><br />In step 2 you say to find this:<br /><pre><font class="small">code:</font><hr> if (!$ThreadRating) { $ThreadRating = "&nbsp;"; }<br /><br /> // Setup the postrow array</pre><hr><br /><br />and your hack to show the word "sticky" on sticky notes begins below that first line. Should the sticky code remain below the first line, and before the rest of this hack, or go elsewhere?

Hi Lisa <img src="/forum/images/icons/smile.gif" alt="" /><br /><br />You can leave the "sticky post" code where it is and add this new code right below it and everything should work.<br /><br />Let me know if it doesn't so I can loose some sleep over it... lol <img src="/forum/images/icons/wink.gif" alt="" />

There are some cases where the "last post by" information isn't being updated when a reply is deleted. Below is what I believe is the fix. Let me know otherwise.. <img src="/forum/images/icons/wink.gif" alt="" /><br /><br />In the instructions change this:<br /><br /><pre><font class="small">code:</font><hr><br />Step 7.<br /> In your deletepost.php script find this:<br /><br /> <br /> // ------------------------------------------------------------<br /> // We also need to decrease the number of replies in this topic<br /> $query = " <br /> UPDATE w3t_Posts<br /> SET B_Replies = B_Replies -1<br /> WHERE B_Number = '$Main'<br /> ";<br /> $dbh -> do_query($query);<br /><br /> <br /> and change it to this:<br /><br /> <br /> $query = "<br /> SELECT B_Number,B_Username,B_Approved<br /> FROM w3t_Posts<br /> WHERE B_Board = '$Board_q'<br /> AND B_Approved = 'yes'<br /> AND B_Main = '$Main'<br /> AND B_Main <> B_Number<br /> ORDER BY B_Number DESC<br /> LIMIT 1<br /> ";<br /> $lastPost = $dbh -> do_query($query); <br /> list($lastNumber,$lastName,$lastApproved) = $dbh -> fetch_array($lastPost);<br /> $lastName = addslashes($lastName);<br /><br /> // ------------------------------------------------------------<br /> // We also need to decrease the number of replies in this topic<br /> $query = " <br /> UPDATE w3t_Posts<br /> SET B_Replies = B_Replies -1,<br /> B_Last_Number = '$lastNumber',<br /> B_Last_Name = '$lastName',<br /> B_Last_Approved = '$lastApproved' <br /> WHERE B_Number = '$Main'<br /> ";<br /> $dbh -> do_query($query); <br /><br /></pre><hr><br /><br />to this:<br /><br /><pre><font class="small">code:</font><hr><br />Step 7.<br /> In your deletepost.php script find this:<br /><br /> <br /> // ------------------------------------------------------------<br /> // We also need to decrease the number of replies in this topic<br /> $query = " <br /> UPDATE w3t_Posts<br /> SET B_Replies = B_Replies -1<br /> WHERE B_Number = '$Main'<br /> ";<br /> $dbh -> do_query($query);<br /><br /> <br /> and change it to this:<br /><br /> <br /> $query = "<br /> SELECT B_Main,B_Number,B_Username,B_Approved<br /> FROM w3t_Posts<br /> WHERE B_Board = '$Board_q'<br /> AND B_Approved = 'yes'<br /> AND B_Main = '$Main'<br /> ORDER BY B_Number DESC<br /> LIMIT 1<br /> ";<br /> $lastPost = $dbh -> do_query($query); <br /> list($lastMain,$lastNumber,$lastName,$lastApproved) = $dbh -> fetch_array($lastPost);<br /> $lastName = addslashes($lastName);<br /> if ($lastMain == $lastNumber) {<br /> // It's the main post therefore no replies left, right?<br /> $lastNumber = "";<br /> $lastName = "";<br /> $lastApproved = "";<br /> }<br /><br /> // ------------------------------------------------------------<br /> // We also need to decrease the number of replies in this topic<br /> $query = " <br /> UPDATE w3t_Posts<br /> SET B_Replies = B_Replies -1,<br /> B_Last_Number = '$lastNumber',<br /> B_Last_Name = '$lastName',<br /> B_Last_Approved = '$lastApproved' <br /> WHERE B_Number = '$Main'<br /> ";<br /> $dbh -> do_query($query); <br /><br /></pre><hr><br /><br />This may or may not be the cure but it is fixing a flaw in the code none the less. <img src="/forum/images/icons/smile.gif" alt="" />

Is it me?<br /><br />Or is there no change in step 6?<br /><br /><pre><font class="small">code:</font><hr><br />Step 6.<br /> In your viewunapproved.php script find this:<br /><br /><br /> // ----------------------------------------<br /> // Grab the unapproved posts for this board <br /> $Keyword_q = addslashes($Keyword);<br /> $query = "<br /> SELECT B_Number,B_Subject,B_Username,B_Posted <br /> FROM {$config['tbprefix']}Posts <br /> WHERE B_Approved = 'no'<br /> AND B_Board = '$Keyword_q'<br /> ORDER BY B_Posted DESC<br /> ";<br /><br /> <br /> and change it to this:<br /><br /> <br /> // ----------------------------------------<br /> // Grab the unapproved posts for this board <br /> $Keyword_q = addslashes($Keyword);<br /> $query = "<br /> SELECT B_Number,B_Subject,B_Username,B_Posted <br /> FROM {$config['tbprefix']}Posts <br /> WHERE B_Approved = 'no'<br /> AND B_Board = '$Keyword_q'<br /> ORDER BY B_Number DESC<br /> "; <br /></pre><hr>

It seems that the ORDER BY is changed from B_Posted to B_Number.

<img src="/forum/images/icons/tongue.gif" alt="" /><br /><br />I need some sleep. <br /><br /><br />Thx.

Hmmmm....<br /><br />I may need more sleep.<br /><br />I've been setting this up on a test board, on 6.1<br /><br />I don't know if I missed a step, but sometimes the last post link is wrong.<br /><br />If this is the link:<br /><br />/showflat.php?Cat=&Board=test&Number=56454&page=0&view=collapsed&sb=5&o=&fpart=1#Post56454<br /><br />Sometimes it's like this:<br /><br />/showflat.php?Cat=&Board=test&Number=56454&page=0&view=collapsed&sb=5&o=&fpart=2#Post56454<br /><br />Note the part=2 instead of 1... if I change the link it works fine.<br /><br />I thought it might be my quick reply mod... but no, I can't seem to find any consistency... <br /><br />I've updated for 6.1 with 'theprefix' variables... but something isn't quite right.<br /><br /><br />Dave, <br />Any idea if this bit of code needs to be tweaked for 6.1???<br /><br />Thx.

I'll have a look before lunch. <img src="/forum/images/icons/smile.gif" alt="" />

I'll have to set this up somewhere so I can test it. I can't seem to see what may be going wrong. It's been working fine on 6.0.x hasnt it?

I had never used it.<br /><br />But I've also got the quick reply mod. But quick reply still uses addpost.php so that shouldn't have any effect.<br />Did some testing here last night and all seemed fine with it.<br /><br />It was pretty simple to install.. and the problem would be with postlist if anything... based on what's happening.<br /><br />Of course I've got a few other mods. Maybe I need to set this up on a clean 6.1 and see if I have the same trouble.

Hey Dave, <br />Before you go to any trouble... let me ask you this:<br /><br />Is the part= calculated in postlist.php? It's not pulled from the database is it? I don't really understand what this is or means.<br /><br />In other words... if it's calculated in postlist.php, then my other scripts should be fine... and I should start with a Clean postlist.php and add things back in to see if it behaves. <img src="/forum/images/icons/tongue.gif" alt="" />

fpart is calculated and not pulled from any database. <img src="/forum/images/icons/smile.gif" alt="" /><br /><br />It is the page the post is on. If you look at a post link with more than one page of replies you can mouse over the number links and see that fpart changes according to the page. 6.1's postlist may be calculating this at some other point but I can't say for sure.

Thanks.<br /><br />I just figured this out.<br /><br />The link wasn't working for the First Post (with a last post link) at the top of postlist because I had a sticky post with no "last post" info above it. once I either unstuck it... or did a reply to the sticky, then all seemed to work.<br />I guess it didn't like having a post in front of it without the last post data. <br /><br />Either way... this hiccup would work itself out as the data is populated when people make replies. <br /><br />Thanks for your help. <br />I will post tweaked instructions for use with .threads 6.1 shortly.

Ok sounds good. <img src="/forum/images/icons/smile.gif" alt="" /><br /><br />Yeah there will be no "last post by" information till someone actually makes a reply to the main post.

Right... and it doesn't like one without the info begin above one with the info. Just went live with it and it all seems to work. I replied to all my sticky posts to avoid the issue.<br /><br />Thanks!

Attached to this post is a version and instructions to use this hack with .threads 6.1.<br /><br />There were a few minor changes. The Database 'theprefix' variable...and postlist called a few more things to use the disk icon etc...<br />So I've adjusted the instructions and added the prefix to the altertable file and all seems to be working well on my 6.1 br1 site. <br /><br />

Thanks Josh <img src="/forum/images/icons/smile.gif" alt="" />

I should have reiterated Loudly...<br />That I take no credit for such a creation other than a few very simple tweaks for ver 6.1.<br /><br />It's All JustDave's wonderful work. <img src="/forum/images/icons/smile.gif" alt="" /><br /><br />I shall do the menial tweaking while you create more cool stuff like IIP. <img src="/forum/images/icons/wink.gif" alt="" />

Which hack is it that inserts the last post by info on the ubbthreads.php page? The Forum Index, like this site has.

I think it's part of the templates or something like that. Allen would know though. I don't much care for the increased queries it creates though.

Ok -<br /><br />This hack worked fine on the beta releases of 6.1... but there was some bug fixing to the last post date and time for sticky threads.<br /><br />There is a bug in 6.1 final release of .threads. If you reply to a sticky thread... the parent post in the thread takes on the date and time of the latest thread.<br /><br />Somehow, a combo of that bug and this hack.... makes every reply in the sticky thread take on the date and time of the latest reply.<br /><br />So I'd wait to install this hack on 6.1 final until Rick posts a fix for the last post time/date problem with sticky threads.... then hopefully we can tweak this accordingly.<br /><br /><img src="/forum/images/graemlins/smile.gif" alt="" />

To fix the wrong time being set on the sticky post, in your addpost.php script find this:<br /><br /><br /><pre><font class="small">code:</font><hr><br />// -------------------------------------------------------<br />// Otherwise just update the reply total if it is approved<br /> elseif ( ($Main) && ($Sticky) && ($Approved == "yes") ) {<br /> $query = "<br /> UPDATE {$config['tbprefix']}Posts<br /> SET B_Replies = B_Replies + 1,<br /> B_Sticky = '$date'<br /> WHERE B_Number = '$Main'<br /> AND B_Board = '$Board_q'<br /> ";<br /> $dbh -> do_query($query);<br /> }<br /><br /></pre><hr><br /><br />and replace it with this:<br /><br /><pre><font class="small">code:</font><hr><br />// -------------------------------------------------------<br />// Otherwise just update the reply total if it is approved<br /> elseif ( ($Main) && ($Sticky) && ($Approved == "yes") ) {<br /> $query = "<br /> UPDATE {$config['tbprefix']}Posts<br /> SET B_Replies = B_Replies + 1,<br /> WHERE B_Number = '$Main'<br /> AND B_Board = '$Board_q'<br /> ";<br /> $dbh -> do_query($query);<br /> }<br /></pre><hr><br /><br /><br />And that *should* do the trick...

So this works correctly with 6.1 then?

I don't know for sure. It's not what's being used here... lol<br /><br />I'll put the "how to" info together sometime this week for the one in use here. <img src="/forum/images/graemlins/smile.gif" alt="" />

I will try the bug fix and test it out....<br /><br />I'll post here if it works correctly. <br />I'm thinking Dave's fix might just do it. <img src="/forum/images/graemlins/wink.gif" alt="" /> <fingers crossed>

If you make JustDave's change.... the Last Post mod will work OK (sorta).<br /><br />Removing that Date code puts back a bug that was present in the beta where the Date and Time in the "Last Post by" column on postlist.php wasn't getting updated for sticky posts.. it remained on the date/time of the first post.<br /><br />Rick fixed it...but in testing we didnt' notice the side effect of this bug fix. <img src="/forum/images/graemlins/tongue.gif" alt="" /><br /><br />My guess is, if you remove that date line of code.... you can use this hack fine. The date and time in the postlist.php for sticky posts will be wrong. The link of this hack will take you to the right post.<br /><br />If you make the fix, install this... we'll watch for Rick's "official" fix to this. <img src="/forum/images/graemlins/tongue.gif" alt="" />

Well I installed the hack and it works fine. I have attached the txt file with a few very minor alterations. JustDave knows of these as you can see from his code above but the hack had not been updated <img src="/forum/images/graemlins/smile.gif" alt="" /><br /><br />The hack calls the Posts database as w3t_Posts<br /><br />In 6.1 it is this {$config['tbprefix']}Posts I simply made those minor fixes. so it is easy to copy and paste. <br /><br />

Actually my 6.1 instructions above are a bit more complete.<br /><br />In 6.1 postlist.php uses $File in the select and list line to tell if there's an attachment and add the disk Icon if needed. <br /><br />Edit: As well as the posterid variable:<br /><br />Step 2.<br /> In your postlist.php script find this:<br /><br /><br /> SELECT t1.B_Number,t1.B_Parent,t1.B_Username,t1.B_Posted,t1.B_Last_Post,t1.B_Subject,t1.B_Main,t1.B_Status,t1.B_Approved,t1.B_Icon, t1.B_Reged,t1.B_Counter,t1.B_Sticky,t1.B_Replies,t1.B_Rating,t1.B_Rates,t1.B_RealRating,t2.U_Color,t2.U_Status,t1.B_PosterId,t1.B_File<br /><br /><br /> and change it to this:<br /><br /> <br /> SELECT t1.B_Number,t1.B_Parent,t1.B_Username,t1.B_Posted,t1.B_Last_Post,t1.B_Last_Number,t1.B_Last_Name,t1.B_Last_Approved,t1.B_Subject,t1.B_Main,t1.B_Status,t1.B_Approved,t1.B_Icon, t1.B_Reged,t1.B_Counter,t1.B_Sticky,t1.B_Replies,t1.B_Rating,t1.B_Rates,t1.B_RealRating,t2.U_Color,t2.U_Status,t1.B_PosterId,t1.B_File<br /><br /> <br /> Find this:<br /><br /> <br /> list($Number,$Parent,$Username,$Posted,$Last_Post,$Subject,$Main,$Open,$Approved,$icon,$Reged,$Counter, $Sticky,$Replies,$Rating,$Rates,$stars,$Color,$PostStatus,$posterid,$file) = $dbh -> fetch_array($sth);<br /><br /><br /> and change it to this:<br /><br /> <br /> list($Number,$Parent,$Username,$Posted,$Last_Post,$Last_Number,$Last_Name,$Last_Approved,$Subject,$Main,$Open,$Approved,$icon,$Reged,$Counter $Sticky,$Replies,$Rating,$Rates,$stars,$Color,$PostStatus,$posterid,$file) = $dbh -> fetch_array($sth);<br /><br /><br />

LOL Yep your right Josh. I found those changes as well but must have forgot to copy them over.

we need to consider breaking the 3 added fields into separate adds. On large forums, the script times out before finishing <img src="/forum/images/graemlins/smile.gif" alt="" />

[]we need to consider breaking the 3 added fields into separate adds. On large forums, the script times out before finishing [/]<br /><br />I'm not sure I understand this.<br />Are you saying on large forums sometimes the last post info doesn't get updated because there's too much stuff to do?<br />Hmmmm.... over my head. I haven't noticed a problem. <knock on wood> But I've probably only got a "medium sized small forum". LOL <img src="/forum/images/graemlins/wink.gif" alt="" />

I'll try and get instructions written for the "last post by" code I have in use here. As you can see there are no extra queries added to the ubbthreads.php display. <img src="/forum/images/graemlins/smile.gif" alt="" /><br /><br />Never enough time... *sigh* <img src="/forum/images/graemlins/crazy.gif" alt="" />

<img src="/forum/images/graemlins/smile.gif" alt="" /> I meant the altertable script that adds the fields to the database originally... <img src="/forum/images/graemlins/smile.gif" alt="" /> If you have a forum with maybe 100k + of posts, the original install of this mod will bring a slower server to its knees <img src="/forum/images/graemlins/wink.gif" alt="" /> and the script times out because it can't finish in 30 seconds.

Hey dave, found any time yet to post the instructions? <img src="/forum/images/graemlins/laugh.gif" alt="" />

I'll try and put it together this weekend. It's all I can do to go through my email and make a quick scan of the web at the end of the day before wanting to crawl into bed. LoL <img src="/forum/images/graemlins/crazy.gif" alt="" />

Totally understandable, keep up the good work and get some rest!

I'm not sure if this bug was covered above.<br /><br />Several hours ago, I moved a thread by msula from the How Do I board to the General Feedback board. It was the "last post" in the How Do I board before I moved it.<br /><br />After the move, the Last Post info for the How Do I board in the Main Index is wrong:<br /><br />10/22/02 12:04 AM by JoshPet: Re: URGENT: How do I change the "forums closed" msg?<br /><br />The URL correctly links to the last post, and JoshPet is the user who made the last post, but the date/time and thread subject correspond to the moved thread, not the current last post.

Thanks for pointing this out. I thought I was going nuts there for a minute.

Actually, it's not a bug. I just hadn't installed the modifications to the scripts in the admin area... lol <img src="/forum/images/graemlins/wink.gif" alt="" /><br /><br />I'll add that to the list I have on my desktop.

Can someone update this hack for 6.1.1?<br />It does not work properly on this version.<br /><br />Regards,<br /><br />Ron.

Be sure to use the updated instructions I posted here.<br /><br />Been working great for me on 6.1.1.<br /><br />What trouble are you having?

Dave:<br /><br />Will this mod work as is in 6.2 or are other changes required?

I am looking into this one as well JJS. I know Dave's answer is NO it will not. There were some major database changes with 6.2 so this hack needs to be tweaked a bit. I know there are quite a few hacks that require some changing. 6.2 implements User Numbers now and not Username so that we could also employ DISPLAY name changes I believe. Since things now use this number scheme I assume that this hack also has to be updated for that.

I'll be adding this back here again so I'll post an update on this when I do. As omegatron stated there are probably some issues to work around now that everything is being based off of the user number and not name.

Yeah, I knew there were extensive updates to this.. I was holding off as I thought Dave had said he had a better way to do this with less queries. <img src="/forum/images/graemlins/smile.gif" alt="" /><br /><br />I will also await updated instructions.

Dave, Chuck---<br /><br />There's seems to be alot of demand for this hack. At least by my users who use the "LAST POST BY" feature to make sure some other member DOESN'T get the last word!! LOL.<br /><br />I'll keep watching for the update.

JJS,<br /><br /> My money is on Dave getting this one out first. <img src="/forum/images/graemlins/wink.gif" alt="" /> Josh way write up some better documentation.<br /><br /> This is one of those hacks that should be a feature of threads and isn't however in the future we shall see. Like Josh said Dave has a better way to do this with less queries.<br /><br />

He he.... yeah for the complicated ones JustDave's the man and I can document. <img src="/forum/images/graemlins/grin.gif" alt="" /> If I get time I'll take a look and see if I can update this, but my plate is overflowing. <img src="/forum/images/graemlins/tongue.gif" alt="" />

plate any mo betta? <img src="/forum/images/graemlins/tongue.gif" alt="" />

I miss this on my board, and here. <img src="/forum/images/graemlins/tongue.gif" alt="" /><br /><br />I'll see if I can get this done tonight. <img src="/forum/images/graemlins/smile.gif" alt="" />

muchas gracias <img src="/forum/images/graemlins/smile.gif" alt="" />

OK - I updated and posted here.<br /><br />We should close this thread please and continue new discussion with the latest version. <img src="/forum/images/graemlins/wink.gif" alt="" />